Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support, tests and documentation for Gentoo #335

Merged
merged 1 commit into from
Oct 12, 2016

Conversation

optiz0r
Copy link
Contributor

@optiz0r optiz0r commented Oct 8, 2016

No description provided.

:id => 'root',
:kernel => 'Linux',
:osfamily => 'Gentoo',
:operatingsystem => 'Gentoo',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you must specify :lsbdistcodename here for TravisCI to be happy.

https://travis-ci.org/stankevich/puppet-python/jobs/166102244

@ghoneycutt
Copy link
Member

Thanks for adding Gentoo support. The code looks good and once the tests are passing, this should be able to be merged.

@optiz0r
Copy link
Contributor Author

optiz0r commented Oct 10, 2016

Thanks @ghoneycutt, I saw that it had failed but didn't yet find the time to investigate, thanks for pointing me directly at the fix. I've pushed this change (lsbdistcodename shows up as "n/a" for me on my system, so that's what I've used for the test). Looks like travis-ci timed out on all the ruby-2.3.1 builds, but working for all previous releases. I don't think that's a result of my code changes? The output seems to stop while running the Debian tests.

@ghoneycutt
Copy link
Member

Yeah, that's not related to this. Could you please squash your commits and I'll do a merge and release tomorrow.

@optiz0r
Copy link
Contributor Author

optiz0r commented Oct 11, 2016

@ghoneycutt squashed and hopefully ready to go!

@ghoneycutt ghoneycutt merged commit 31e68a9 into voxpupuli:master Oct 12, 2016
@ghoneycutt
Copy link
Member

Thank you!

Released in 1.18.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants